Skip to content

Conversation

@jgmcalpine
Copy link
Contributor

@jgmcalpine jgmcalpine commented Feb 3, 2026

Updates rust-lightning dependencies to 817ab5e583002df5e32b3a71e7ab093005a2a39a.

This fixes the build breakage resulting from the InvoiceBuilder and PaymentHash refactor in upstream PR #4363.

Specific changes:

  • Updates Cargo.toml revision.
  • Wraps Sha256 hash in PaymentHash in src/liquidity.rs to match the new InvoiceBuilder API.

@ldk-reviews-bot
Copy link

ldk-reviews-bot commented Feb 3, 2026

I've assigned @tnull as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

@ldk-reviews-bot ldk-reviews-bot requested a review from tnull February 3, 2026 18:16
Copy link
Collaborator

@tnull tnull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! You'll need to bump the bitcoin-payment-instructions dependency also to make this build.

Cargo.toml Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please bump this to e4d519b95b26916dc6efa22f8f1cc11a818ce7a7 in order to have the bitcoin-payment-instructions dependency use the same version of lightning.

@jgmcalpine jgmcalpine force-pushed the fix-invoice-builder-breakage branch from 13eadf3 to 1ffc0d2 Compare February 4, 2026 13:53
Copy link
Collaborator

@tnull tnull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@tnull tnull merged commit 80fb49b into lightningdevkit:main Feb 4, 2026
17 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants